Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

East Sale Procedures #288

Merged
merged 1 commit into from
Aug 21, 2024
Merged

East Sale Procedures #288

merged 1 commit into from
Aug 21, 2024

Conversation

glennawatson
Copy link
Contributor

@glennawatson glennawatson commented Aug 14, 2024

Summary

Adds East Sale ADC and TCU pages.

Changes

Additions:

  • East Sale aerodrome and terminal area procedures

Copy link
Contributor

@alphadelta332 alphadelta332 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work again Glenn, thanks for your efforts. Just left a few comments. Cheers

docs/aerodromes/classc/EastSale.md Outdated Show resolved Hide resolved
docs/aerodromes/classc/EastSale.md Outdated Show resolved Hide resolved
docs/aerodromes/classc/EastSale.md Outdated Show resolved Hide resolved
docs/aerodromes/classc/EastSale.md Outdated Show resolved Hide resolved
docs/aerodromes/classc/EastSale.md Outdated Show resolved Hide resolved
docs/aerodromes/classc/EastSale.md Show resolved Hide resolved
docs/aerodromes/classc/EastSale.md Outdated Show resolved Hide resolved
docs/aerodromes/classc/EastSale.md Outdated Show resolved Hide resolved
docs/aerodromes/classc/EastSale.md Show resolved Hide resolved
docs/aerodromes/classc/EastSale.md Outdated Show resolved Hide resolved
@mattkelly4
Copy link
Collaborator

Looking great @glennawatson , thanks again for your efforts. Just some small tweaks/clarifications from my end.

@glennawatson
Copy link
Contributor Author

glennawatson commented Aug 17, 2024

Looking great @glennawatson , thanks again for your efforts. Just some small tweaks/clarifications from my end.

@mattkelly4 That taxi graph for YSSY might have to replaced, looks to be the ASA version of the taxi map rather than VATSYS so might be copyright issues.

@mattkelly4
Copy link
Collaborator

Looking great @glennawatson , thanks again for your efforts. Just some small tweaks/clarifications from my end.

@mattkelly4 That taxi graph for YSSY might have to replaced, looks to be the ASA version of the taxi map rather than VATSYS so might be copyright issues.

Yeah good idea, I'll open an issue. Thanks

@glennawatson
Copy link
Contributor Author

Alright, given some have asked things to move into the TCU page, I've done a initial attempt at doing the TCU page as well.

@mattkelly4 @alphadelta332 you may want to review that page as well.

Images are likely to come tomorrow, want to get images for the training areas, WSL, ESL, the inbound/outbound lanes, and the taxi diagram mentioned, and the VFR transit lanes princes, wsl, and the longreach ones.

@glennawatson
Copy link
Contributor Author

glennawatson commented Aug 17, 2024

Question, technically since TCU controls the circuit area I've replicated the circuit information in both pages, I suspect 90% of the time the TCU will offload to a tower if they are on.

WSL circuit area I've not included in the ES ADC page, since that doesn't really make sense to the ES controller though.

Is this what you guys want?

@mattkelly4
Copy link
Collaborator

Alright, given some have asked things to move into the TCU page, I've done a initial attempt at doing the TCU page as well.

@mattkelly4 @alphadelta332 you may want to review that page as well.

Images are likely to come tomorrow, want to get images for the training areas, WSL, ESL, the inbound/outbound lanes, and the taxi diagram mentioned, and the VFR transit lanes princes, wsl, and the longreach ones.

Unreal, thanks so much Glenn, really appreciate the efforts.

@mattkelly4
Copy link
Collaborator

Question, technically since TCU controls the circuit area I've replicated the circuit information in both pages, I suspect 90% of the time the TCU will offload to a tower if they are on.

WSL circuit area I've not included in the ES ADC page, since that doesn't really make sense to the ES controller though.

Is this what you guys want?

Yeah that sounds great. It is expected that TCU will release the circuit area to ADC, but it's helpful to know the various circuit heights, etc so that both controllers are aware of the extent of the release. Having the info in both places sounds like a good way of doing that.

Copy link
Contributor

@alphadelta332 alphadelta332 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great stuff again Glenn. Just a few more comments on the new content. Thanks for your work

docs/aerodromes/classc/EastSale.md Outdated Show resolved Hide resolved
docs/terminal/eastsale.md Outdated Show resolved Hide resolved
docs/terminal/eastsale.md Outdated Show resolved Hide resolved
docs/terminal/eastsale.md Outdated Show resolved Hide resolved
docs/terminal/eastsale.md Show resolved Hide resolved
@glennawatson glennawatson changed the title Initial ESL ADC Initial ESL ADC/TCU Aug 17, 2024
@glennawatson
Copy link
Contributor Author

@mattkelly4 @alphadelta332 The images and diagrams are in.

Copy link
Contributor

@alphadelta332 alphadelta332 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just small finishing touches now. Hope you don't think it's too nitpicky haha, just making sure its all nice before publishing. Thanks mate

docs/aerodromes/classc/EastSale.md Outdated Show resolved Hide resolved
docs/aerodromes/classc/EastSale.md Outdated Show resolved Hide resolved
Copy link
Contributor

@alphadelta332 alphadelta332 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And just standby on pushing that fix, we'll wait to hear from @zkgell on the ES APP login identifier (Talking about the ESL, not the ES_APP). I have messaged him directly and will advise when I get a response

@zkgell
Copy link
Contributor

zkgell commented Aug 19, 2024

East Sale Approach internal position ID is ESA

docs/terminal/eastsale.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@mattkelly4 mattkelly4 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is looking great Glenn, and those diagrams are mint! Well done. Just a few more minor changes and we should be right to publish

docs/terminal/eastsale.md Outdated Show resolved Hide resolved
docs/terminal/eastsale.md Outdated Show resolved Hide resolved
docs/terminal/eastsale.md Show resolved Hide resolved
docs/terminal/eastsale.md Show resolved Hide resolved
docs/terminal/eastsale.md Outdated Show resolved Hide resolved
alphadelta332
alphadelta332 previously approved these changes Aug 20, 2024
@glennawatson
Copy link
Contributor Author

Alright, just got the formatting working for the hotline/radio comms work, @alphadelta332 and @mattkelly4 finished with the changes again.

@alphadelta332 alphadelta332 requested a review from a team August 20, 2024 12:15
@mattkelly4 mattkelly4 changed the title Initial ESL ADC/TCU East Sale Procedures Aug 21, 2024
@tylerthetiletiler tylerthetiletiler merged commit 7030b8f into vatpac-technology:main Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants